Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CURATOR-470] Add new service cache listener to let users know what instances have changed #268

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Randgalt
Copy link
Member

@Randgalt Randgalt commented Jun 24, 2018

Built off of #265

A new interface (ServiceCacheEventListener) is introduced to expose changed instances, while preserving backward compatibility.

Unit test has updated as well.

@JerryChin this PR builds on yours but is more backward compatible. I'm concerned that the new listener breaks the contract of the old one. So, the cache now maintains two separate listener containers.

JerryChin and others added 4 commits June 8, 2018 13:06
…changed, while preserving backward compatibility.
Introduces an alternate cache listener, ServiceCacheEventListener, that gives more detail about changes to the cache as opposed to the original version which merely denotes a change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants